-
-
Notifications
You must be signed in to change notification settings - Fork 44
feat: いい感じ変換の設定をMagic Conversionウィンドウに統合 #265
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
base: main
Are you sure you want to change the base?
Conversation
…View Move all いい感じ変換 (Magic Conversion) settings to the prompt input window: - Add settings button (gear icon) to Magic Conversion window header - Create MagicConversionSettingsSheet with keyboard shortcut, AI backend, and OpenAI API settings - Remove いい感じ変換 section from ConfigWindow basic tab - Remove 基本ショートカット section from ConfigWindow customize tab - Clean up unused variables and functions from ConfigWindow Co-Authored-By: Claude Opus 4.5 <[email protected]>
Co-Authored-By: Claude Opus 4.5 <[email protected]>
Co-Authored-By: Claude Opus 4.5 <[email protected]>
…ortcuts-in-pin-ui
Keep AI backend settings in ConfigWindow for easier access, while shortcuts remain configurable in PromptInputView. Co-Authored-By: Claude Opus 4.5 <[email protected]>
- Remove debug logger and timing measurements from InputController - Add early return optimization for custom prompt shortcut check - Remove "Created by Claude Code" file headers Co-Authored-By: Claude Opus 4.5 <[email protected]>
…ortcuts-in-pin-ui # Conflicts: # azooKeyMac/InputController/azooKeyMacInputController.swift
Co-Authored-By: Claude Opus 4.5 <[email protected]>
…ortcuts-in-pin-ui
- Remove debug print statements from KeyboardShortcutConfigItem - Remove debug messages from custom shortcut handling - Add custom decoder to PromptHistoryItem for backward compatibility with existing data Co-Authored-By: Claude Opus 4.5 <[email protected]>
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
このファイルなんだけど、Apple Platform非依存のロジックに関してはCore/側で管理していきたいので、NSEvent部分以外を上手くCore/の方に書いてほしい。
| @@ -0,0 +1,74 @@ | |||
| @_spi(Core) import Core | |||
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
@_spi(Core)って何……?
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
このファイルも同じ理由で、実装可能な部分はなるべくCore/側に置いてほしい。今はほとんどのConfigはCore/側に移動してる。
| import Cocoa | ||
|
|
||
| /// キーボードショートカットを表す構造体 | ||
| public struct KeyboardShortcut: Codable, Equatable, Hashable, Sendable { |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
KeyboardShotrcut側にeisuDoubleTapとkanaDoubleTapを入れる案はない?
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
入れてもいいんだけど将来的になんか困りそうじゃない?そうでもないか?
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
うーん?具体的なシナリオってある?
| return | ||
| } | ||
|
|
||
| if event.keyCode == 53 { |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
この辺のキーコード、キーコードで書いてあってもパッと分からないのでコメントを書いて欲しい
概要
変更内容
Magic Conversionウィンドウ
ConfigWindow
コード整理
CustomPromptShortcut構造体とCustomPromptShortcutsEditor.swiftを削除PromptHistoryItemに直接保存するように変更テスト項目
🤖 Generated with Claude Code